Skip to content

[jylee2033] WEEK 01 solutions#2361

Open
jylee2033 wants to merge 1 commit intoDaleStudy:mainfrom
jylee2033:main
Open

[jylee2033] WEEK 01 solutions#2361
jylee2033 wants to merge 1 commit intoDaleStudy:mainfrom
jylee2033:main

Conversation

@jylee2033
Copy link

@jylee2033 jylee2033 commented Mar 2, 2026

답안 제출 문제

작성자 체크 리스트

  • Projects의 오른쪽 버튼(▼)을 눌러 확장한 뒤, Week를 현재 주차로 설정해주세요.
  • 문제를 모두 푸시면 프로젝트에서 StatusIn Review로 설정해주세요.
  • 코드 검토자 1분 이상으로부터 승인을 받으셨다면 PR을 병합해주세요.

검토자 체크 리스트

Important

본인 답안 제출 뿐만 아니라 다른 분 PR 하나 이상을 반드시 검토를 해주셔야 합니다!

  • 바로 이전에 올라온 PR에 본인을 코드 리뷰어로 추가해주세요.
  • 본인이 검토해야하는 PR의 답안 코드에 피드백을 주세요.
  • 토요일 전까지 PR을 병합할 수 있도록 승인해주세요.

@@ -0,0 +1,6 @@
class Solution:
def twoSum(self, nums: List[int], target: int) -> List[int]:
for i in range(len(nums) - 1):
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

python 스타일 관점에서 작은 제안 하나 드립니다.
현재 코드는 nums[i]를 반복해서 접근하는 구조인데, enumerate를 사용하면 조금 더 pythonic하고 읽기 쉬운 코드로 표현할 수 있습니다!

예를 들면 아래와 같은 형태입니다.

for i, a in enumerate(nums):
    for j in range(i + 1, len(nums)):
        if a + nums[j] == target:
            return [i, j]

이렇게 하면 nums[i] 반복 접근을 줄일 수 있고,
a라는 변수로 현재 값을 바로 표현할 수 있어서 코드 의도가 조금 더 명확해집니다.

하지만 지금 코드도 충분히 직관적이고 좋은 코드라고 생각합니다!

Copy link
Contributor

@riveroverflows riveroverflows left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

안녕하세요! 리뷰가 너무 늦었네요 죄송합니다.

문제를 직관적인 방식으로 깔끔하게 구현하셔서 전체적인 코드 흐름을 읽기 좋았습니다!

이번주 고생 많으셨습니다.
좋은 주말 보내시고 다음주도 파이팅입니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Solving

Development

Successfully merging this pull request may close these issues.

2 participants